Code Review

This pull request introduces the MLSDictionaryFeature module, which implements the main dictionary views, detail views for items, monsters, maps, NPCs, and quests, search functionality, and notification management. The code reviewer identified several critical and high-severity issues, particularly regarding Rx execution timing and potential runtime crashes. Specifically, dynamically appending to contentViews based on asynchronous network responses can cause index out-of-bounds crashes, and several Rx operations (such as addRecentSearch and CheckLoginUseCase.execute) are executing synchronously outside the stream instead of being deferred to subscription time. Additionally, pagination logic in scroll views lacks proper loading state guards or timestamp updates, which can trigger duplicate network requests.

수고 많으셨습니다!!
모듈 이전은 diff가 너무커서 pr을 확인하기가 어렵네요 ㅎㅎ..
AI로 받은 리뷰내용만 공유 드립니다! 참고만 부탁드려요!

🔴 Critical

1. navigatTo 오타 — DictionaryListReactor, ItemDictionaryDetailReactor
   Mutation case 이름이 navigateTo가 아니라 navigatTo로 되어있음. 오타인데 전체적으로 통일돼있어서 빌드는 되지만, 다른 Reactor들과 이름이 달라서 검색/추적이 어려움.

2. addRecentSearch — Rx 지연 실행 원칙 위반 (RecentSearchRepositoryImpl)
   Completable을 반환하지만 UserDefaults 쓰기가 구독 시점이 아니라 함수 호출 시점에 즉시 실행됨. 구독 전에 side effect가 발생. Completable.create 안으로 감싸야 함. (removeRecentSearch는 올바르게 감싸져 있는데
   addRecentSearch만 빠져있음)

3. DictionaryDetailFactoryImpl — 자기 자신 순환 참조
   var detailFactory: DictionaryDetailFactoryImpl!
   detailFactory = DictionaryDetailFactoryImpl(
   dictionaryDetailFactory: { detailFactory } // strong 캡처
   )
   detailFactory가 자신을 strong capture하는 클로저를 들고 있어 retain cycle. SceneDelegate에서도 동일하게 구성됨. [weak detailFactory]로 약한 참조하거나 구조 재검토 필요.

🟠 High

4. Index out of bounds — MonsterDictionaryDetailReactor
   return .just(.navigateTo(.detail(type: .item, id: currentState.dropItems[index].itemId)))
   배열 범위 체크 없이 index로 직접 접근. 빠른 탭이나 비동기 타이밍 이슈로 crash 가능.

5. Info.plist에 UILaunchStoryboardName 누락
   LaunchScreen.storyboard를 추가했지만 Info.plist에 키가 없어서 iOS가 인식 못 함. 결과적으로 앱이 letterbox(호환성) 모드로 실행되어 풀스크린이 아닌 검은 여백 발생.

6. DictionaryDetailBaseViewController — firstStickyIndexButton 잘못된 할당
   if index == 0 {
   firstIndexButton = button
   firstStickyIndexButton = button // ← stickyButton을 넣어야 하는데 button이 들어감
   }
   stickyButton용 변수에 일반 button이 들어가는 버그.

7. MLSDictionaryFeature가 MLSMyPageFeature에 의존
   Package.swift를 보면 MLSDictionaryFeature 타깃이 MLSMyPageFeatureInterface에 의존하고 있음. 도감 기능이 마이페이지 모듈에 의존하는 건 모듈 경계가 흐릿해지는 것. fetchProfileUseCase 때문인데, 공통 Interface
   계층으로 빼거나 DictionaryFeatureInterface에 자체 프로토콜 정의가 필요함.

🟡 Medium

8. MLSMyPageFeatureTesting import — Example 앱
   MLSDictionaryFeatureExample이 MLSMyPageFeatureTesting을 import하고 있음. Dictionary Example이 MyPage Testing에 의존할 이유가 없음. 해당 Mock들을 MLSDictionaryFeatureTesting에 직접 두는 게 맞음.

9. ViewController.swift 파일 삭제 필요
   // SceneDelegate에서 직접 DictionaryMainViewController를 띄우기 때문에 사용하지 않습니다.
   class ViewController: UIViewController {}
   주석으로 설명하고 파일 살려두는 것보다 그냥 삭제하는 게 맞음.

10. 매직 넘버 상수화 — DictionaryListViewController
    guard now.timeIntervalSince(lastPagingTime) > 0.5 else { return }
    if offsetY > contentHeight - height - 100 {
    0.5, 100 모두 상수로 빼야 의도가 명확해짐.

11. 테스트 누락 케이스

• toggleBookmark error case (bookmarkError route)
• undoLastDeletedBookmark 성공/실패
• pagination 연속 호출
• itemFilterOptionSelected

🟢 Low

12. currentState vs self.currentState 혼용
    DictionaryListReactor에서 같은 메서드 내에 self.currentState와 currentState가 혼용됨. 동작은 같지만 스타일 일관성이 떨어짐.